Skip to content

Conversation

@roomote
Copy link
Contributor

@roomote roomote bot commented Aug 1, 2025

This PR addresses issue #6557 by adding comprehensive documentation for Docker MCP Gateway streaming transport configuration.

Summary of Changes

  • Created docs/docker-mcp-gateway-streaming.md with detailed documentation explaining:

    • The issue users face when connecting to Docker MCP Gateway with streaming transport
    • The root cause (missing /mcp path in the URL)
    • Clear examples of correct vs incorrect configuration
    • Quick reference table for transport types and URL formats
  • Updated README.md to include a reference to the new documentation in the MCP section

Problem Solved

Users were experiencing "Invalid session ID" errors when trying to connect to Docker MCP Gateway using the streaming transport because they were using the root URL (e.g., http://localhost:8080) instead of the required /mcp endpoint (e.g., http://localhost:8080/mcp).

This documentation clarifies the correct URL format and helps prevent future confusion.

Fixes #6557


Important

Adds documentation for Docker MCP Gateway streaming transport configuration, addressing URL path issues.

  • Documentation:
    • Adds docs/docker-mcp-gateway-streaming.md with detailed instructions on configuring Docker MCP Gateway with streaming transport.
    • Explains the issue of using the root URL instead of /mcp, causing "Invalid session ID" errors.
    • Provides correct vs incorrect configuration examples and a quick reference table.
  • README Update:
    • Adds a note in README.md under the MCP section to reference the new documentation for Docker MCP Gateway streaming transport.

This description was created by Ellipsis for 3985653. You can customize this summary. It will automatically update as commits are pushed.

- Created comprehensive documentation for Docker MCP Gateway streaming transport configuration
- Clarified that the /mcp path is required when using streaming transport mode
- Added reference in main README.md to guide users to the new documentation
- Addresses issue #6557 where users were experiencing connection errors due to missing /mcp path
@roomote roomote bot requested review from cte, jr and mrubens as code owners August 1, 2025 16:37
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. documentation Improvements or additions to documentation labels Aug 1, 2025
Copy link
Contributor Author

@roomote roomote bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewing my own code is like grading my own homework - suspiciously generous but still finding issues.

When configuring your MCP client to connect to a Docker MCP Gateway running with streaming transport, you must include the `/mcp` path in the URL:

```json
{
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The JSON examples use tabs for indentation, which is inconsistent with typical JSON formatting standards. Consider using 2 or 4 spaces instead:


## Additional Notes

- This requirement is specific to the `streaming` transport mode
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this path requirement specific only to the streaming transport? It would be helpful to clarify whether other transport modes have different requirements or if this applies universally.


## Quick Reference

| Transport Type | Correct URL Format |
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The quick reference table only shows the streaming transport. Consider either:

  1. Adding other transport types with their URL formats, or
  2. Explicitly stating that this requirement is unique to streaming transport

This would make the documentation more comprehensive.

## Related Links

- [MCP Documentation](https://docs.roocode.com/advanced-usage/mcp)
- [Docker MCP Toolkit Documentation](https://docs.roocode.com/advanced-usage/mcp#docker-mcp-toolkit)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider adding a troubleshooting section with common error messages users might encounter (like "Invalid session ID") and their solutions. This would help users debug issues more effectively.


## Issue

When connecting an MCP client to the Docker MCP Gateway using the `streaming` transport (`docker mcp gateway run --transport streaming`), the connection will fail with errors like "Invalid session ID" if the client is configured to connect to the root URL (e.g., `http://localhost:8080`).
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The document mentions but doesn't show a complete example. Would it be helpful to add a full command example with all necessary parameters for starting the gateway?

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Aug 1, 2025
@daniel-lxs
Copy link
Member

This is related to the marketplace, closing

@daniel-lxs daniel-lxs closed this Aug 2, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Aug 2, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Aug 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[Docs] Docker MCP Toolkit | the 'streaming' transport requires the /mcp URL path

4 participants